Skip to content

Conversation

@stefan-as
Copy link
Contributor

No description provided.

Comment on lines +9 to +11
t.storeSecret(tokenName, window.togglToken.value).then(() => {
t.closePopup();
});

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ich frage mich hier, ob wir für den fall, dass storeSecret nicht erfolgreich ist, nicht wenigstens ein console.log haben wollen.

Copy link

@aboutnick aboutnick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so, alles schön und gut. nur leider kann ich nirgendwo nachvollziehen, warum das jetzt die lösung für markus' problem sein sollte. daher denke ich, die idee ist, hier schnell voranzugehen und es einfach auszuprobieren.

🚀 🎸

@stefan-as
Copy link
Contributor Author

@aboutnick Das Problem entsteht, weil verschiedene Orgas mit unterschiedlichen Schlüsseln auf dasselbe Secret zugreifen. Deswegen sorgt diese Änderung dafür, dass das ein Secret pro Orga wird und nicht mehr ein einziges Secret für alle.

@stefan-as stefan-as merged commit 43d23c7 into main Apr 10, 2025
3 checks passed
@stefan-as stefan-as deleted the fix/8122 branch April 10, 2025 08:05
@aboutnick
Copy link

weil verschiedene Orgas

@stefan-as da gehts schon los. was für verschiedene orgas? mir ist das gesamte scenario unklar. evtl schnattern wir mal, wenn du mal kurz 10 min luft hast. zb an einem freitag.. 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants